Skip to content

[stable34] fix(jobs): don't overwrite the --stop_after baseline in background-job:worker - #63328

Merged
joshtrichards merged 1 commit into
stable34from
backport/63297/stable34
Aug 19, 2026
Merged

[stable34] fix(jobs): don't overwrite the --stop_after baseline in background-job:worker#63328
joshtrichards merged 1 commit into
stable34from
backport/63297/stable34

Conversation

@backportbot

@backportbot backportbot Bot commented Aug 17, 2026

Copy link
Copy Markdown

Backport of PR #63297

…b:worker

$startTime was used for two different things:
as baseline for --stop_after and as start time of the currently running job. 
The variable was reassigned during each loop with each new job in the queue, so the baseline for --stop_after was lost and the worker never terminated.

Renaming the job variable fixes this issue.

Assisted-by: Claude:claude-opus-5

Signed-off-by: Sven Hansen <vaenda@vaenda.de>
@backportbot
backportbot Bot requested a review from a team as a code owner August 17, 2026 21:54
@backportbot
backportbot Bot requested review from Altahrim, CarlSchwan, icewind1991, solracsf and sorbaugh and removed request for a team August 17, 2026 21:54
@backportbot backportbot Bot added bug 3. to review Waiting for reviews AI assisted community pull requests from community labels Aug 17, 2026
@backportbot backportbot Bot added this to the Nextcloud 34.0.4 milestone Aug 17, 2026
@joshtrichards
joshtrichards merged commit dbf904f into stable34 Aug 19, 2026
244 of 252 checks passed
@joshtrichards
joshtrichards deleted the backport/63297/stable34 branch August 19, 2026 00:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews AI assisted bug community pull requests from community

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants